[COMMON] FEAT: 수요지식회 날짜 변경하는 기능 추가 #1702
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
기존 시간설정을 하지 않아 조회 당일 기준보다 이전의 폼을 반환하지 않는 오류 수정
…github.com/innovationacademy-kr/Cabi into common/dev/refactor-presentation-date#1691
…om:innovationacademy-kr/Cabi into common/dev/refactor-presentation-date#1691
…github.com/innovationacademy-kr/Cabi into common/dev/refactor-presentation-date#1691
…om:innovationacademy-kr/Cabi into common/dev/refactor-presentation-date#1691
…om:innovationacademy-kr/Cabi into common/dev/refactor-presentation-date#1691
ableDates -> results
…om:innovationacademy-kr/Cabi into common/dev/refactor-presentation-date#1691
presentation -> dummyForm
3 tasks
jnkeniaem
reviewed
Nov 5, 2024
Comment on lines
+24
to
+26
| // TODO: 추가해야함^^ | ||
| } finally { | ||
| // TODO | ||
| // TODO: 추가해야함^^ |
Comment on lines
+82
to
+86
| if (isAdmin) { | ||
| return true; | ||
| } else { | ||
| return !itemInfo.itemStatus; | ||
| } |
There was a problem hiding this comment.
if문에서 바로 return 하니 else block 없어도 충분하지 않나 싶습니다!
cf) IF ELSE 대신 빠른 RETURN의 코딩 스타일을 사용합시다.
| @@ -196,7 +218,14 @@ const EditStatusModal = ({ closeModal }: EditStatusModalProps) => { | |||
| <ContentSectionStyled> | |||
| <ContentItemSectionStyled> | |||
| <ContentItemWrapperStyled isVisible={true}> | |||
| <ContentItemTitleStyled>발표 상태</ContentItemTitleStyled> | |||
| <CntentItemTitleStyled> | |||
Comment on lines
+224
to
+226
| <ContentItemCancleAlertStyled> | ||
| 발표를 취소하면 되돌릴 수 없습니다 | ||
| </ContentItemCancleAlertStyled> |
| MAX_CONTENT_LENGTH, | ||
| MAX_SUMMARY_LENGTH, | ||
| MAX_TITLE_LENGTH, | ||
| } from "@/Presentation/constants/policy"; | ||
| import { axiosGetPresentationAbleDates } from "../api/axios/axios.custom"; |
|
7차 프론트 첫 정식 프로젝트!! 이제 수지회 날짜 유연하게 변경 가능해서 운영하기 편할것같습니다 모두들 고생하셧슴다 ~~~ |
jnkeniaem
reviewed
Nov 5, 2024
| @@ -99,7 +100,7 @@ const DropdownContainerStyled = styled.div` | |||
| flex-direction: column; | |||
| width: 100%; | |||
| position: relative; | |||
| cursor: pointer; | |||
| cursor: "pointer"; | |||
There was a problem hiding this comment.
큰따옴표가 들어가서 hover 했을때 커서 모양이 안바뀌는것같습니다..!
| @@ -113,14 +114,16 @@ const DropdownSelectionBoxStyled = styled.div` | |||
| text-align: start; | |||
| padding-left: 20px; | |||
| font-size: 1.125rem; | |||
| color: var(--sys-main-color); | |||
| color: "var(--sys-main-color)"; | |||
There was a problem hiding this comment.
큰따옴표가 들어가서 글자가 main color로 안보입니다
| @@ -143,9 +146,8 @@ const DropdownItemStyled = styled.div<{ | |||
| border: 1px solid var(--line-color); | |||
| border-width: 0px 1px 1px 1px; | |||
| width: 100%; | |||
| height: 60px; | |||
There was a problem hiding this comment.
개인적으로 기존처럼 DropdownSelectionBoxStyled, DropdownItemStyled의 height를 맞추는게 UI가 더 괜찮은것같습니다..!
| @@ -113,14 +114,16 @@ const DropdownSelectionBoxStyled = styled.div` | |||
| text-align: start; | |||
| padding-left: 20px; | |||
| font-size: 1.125rem; | |||
| color: var(--sys-main-color); | |||
| color: "var(--sys-main-color)"; | |||
| `; | |||
|
|
|||
| const DropdownItemContainerStyled = styled.div<{ isVisible: boolean }>` | |||
There was a problem hiding this comment.
드롭다운 아이템이 많아서 그런지DropdownItemContainerStyled 아랫 부분 / 윗 부분 이 잘립니다..
여기에 border radius를 설정하면 해결될 것 같습니당
| @@ -99,7 +100,7 @@ const DropdownContainerStyled = styled.div` | |||
| flex-direction: column; | |||
| width: 100%; | |||
| position: relative; | |||
| cursor: pointer; | |||
| cursor: "pointer"; | |||
| `; | |||
|
|
|||
| const DropdownSelectionBoxStyled = styled.div` | |||
There was a problem hiding this comment.
dummy의 경우 발표 상태 선택 못하니까 css로 클릭 막혀있는걸 나타내면 좋을것같습니다.
ex) DropdownSelectionBoxStyled 발표 예정 글씨색을 회색으로
| onClick={() => { | ||
| !itemInfo.itemStatus && handleItemClick(itemInfo.item); | ||
| isClickable && handleItemClick(itemInfo.item); |
There was a problem hiding this comment.
화면 너비가 작아지면 더미는 클릭 안되네용 너비 줄어도 클릭되게 변경해주세요~
jnkeniaem
reviewed
Nov 5, 2024
| <ContentItemTitleStyled>발표 상태</ContentItemTitleStyled> | ||
| {presentationStatus === PresentationStatusType.CANCEL && ( | ||
| <ContentItemCancleAlertStyled> | ||
| 발표를 취소하면 되돌릴 수 없습니다 |
There was a problem hiding this comment.
WarningNotification이라는게 만들어져있습니당 활용하는게 어떠신지..!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
해당 사항 (중복 선택)
설명
[ COMMON ]
FE에서 진행했던 작업을 더미데이터를 활용하여BE에서 작업하도록 변경Admin에서 발표 취소에 대한 정책 변경user_id가null인 것을 기준으로 판별하기로 함 (category에 DUMMY라는 항목이 따로 없음)[ FE ]
BE에서 발표 가능한 날짜를 받도록 로직 변경: 이에 따라 기존의 계산 로직에 대해 사용하지 않는 부분 삭제
Admin에서 발표 취소에 따른 변경user_id가null인 것을 기준으로 판별하기로 함 (전달된 데이터에 category에 DUMMY라는 항목이 따로 없음)Admin에서 지난 날짜에 대해, 발표 날짜를 수정 가능하도록 변경: 일정 관리 모달을 선택할 수 있도록 추가
Admin에서 발표가 신청되지 않은 날짜에 대해, 상태를 수정하지 못하도록 변경: 선택하지 못하도록 수정
Admin에서 시크릿 탭으로(즉, 저장된 캐시가 없는 경우) 소셜 로그인이 아닌 일반 로그인으로 접속 시, 수지회 탭으로 넘어가지 않는 에러 해결: token 로드 위치 조정
일반 유저의 '상세정보' 탭과Admin의 '일정관리' 탭에서 2025년으로 넘어갈 때 발생하는 400에러 및 화살표가 이상하게 나타나는 현상 해결: 1월에 대해 '1'이 아닌 '01'로 데이터가 넘어가도록 수정(400에러 원인)
,
month로 비교했던 부분을year까지 함께 비교하도록 함수 및 로직 추가[ BE ]
#1691